Overview

This PR fixes a bug where Dev Proxy Toolkit incorrectly shows upgrade notifications even when the latest version is already installed. The issue occurred because the version extraction logic was picking up version numbers from file paths instead of actual update notifications.

Problem

When devproxy outdated --short runs, it can output different things depending on the configuration:

When no updates are available:

info    1 error responses loaded from /opt/homebrew/Cellar/dev-proxy/v0.29.1/devproxy-errors.json

When an update is available:

info    1 error responses loaded from /opt/homebrew/Cellar/dev-proxy/v0.29.0/devproxy-errors.json
info    v0.29.1

The original extractVersionFromOutput function used a generic regex that would match the first version number found, which was always from the file path. This caused false upgrade notifications suggesting users upgrade to the same version they already had installed.

Solution

Updated the extractVersionFromOutput function to:

1. Process output line-by-line instead of using a global regex
2. Skip lines containing file paths (identified by /, \, or loaded from)
3. Only extract versions from dedicated notification lines
4. Handle both Unix and Windows file paths correctly

Changes Made

Core Fix

• src/detect.ts: Enhanced extractVersionFromOutput function with path-aware logic
• Added comprehensive comments explaining the fix

Tests Added

• 5 new test cases specifically for issue Dev Proxy Toolkit shows false upgrade notification when no updates available #286 scenarios:
  • No false positives for Unix file paths
  • No false positives for Windows file paths
  • Correct extraction when actual updates are available
  • Mixed scenarios with both file paths and update notifications

Validation

• ✅ All existing tests continue to pass (no regressions)
• ✅ Build and lint successful
• ✅ Production build successful

Before & After

Before (❌ Bug):

// Input: "info 1 error responses loaded from /path/v0.29.1/file.json"
extractVersionFromOutput(input) // Returns "0.29.1" - FALSE POSITIVE!

After (✅ Fixed):

// Input: "info 1 error responses loaded from /path/v0.29.1/file.json"  
extractVersionFromOutput(input) // Returns "" - No false notification

// Input: "info 1 error responses...\ninfo v0.29.1"
extractVersionFromOutput(input) // Returns "0.29.1" - Correct notification

Impact

Users will no longer see misleading upgrade notifications when they already have the latest version. The toolkit now correctly distinguishes between version numbers in file paths versus actual update availability notifications.

Fixes #286.

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.